Skip to content

Conversation

@jkbktl
Copy link
Contributor

@jkbktl jkbktl commented Oct 1, 2025

  • needs to update links and labels, but reviewable

closes #774

@changeset-bot
Copy link

changeset-bot bot commented Oct 1, 2025

🦋 Changeset detected

Latest commit: f204746

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@status-im/status-network Patch
hub Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@jkbktl jkbktl self-assigned this Oct 1, 2025
@vercel
Copy link

vercel bot commented Oct 1, 2025

@jkbktl is attempting to deploy a commit to the Status Team on Vercel.

A member of the Team first needs to authorize it.

@jkbktl jkbktl linked an issue Oct 1, 2025 that may be closed by this pull request
@vercel
Copy link

vercel bot commented Oct 2, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
status-network-hub Ready Ready Preview Comment Oct 22, 2025 0:29am

@jkbktl jkbktl marked this pull request as ready for review October 2, 2025 21:02
@jkbktl jkbktl requested a review from Copilot October 2, 2025 21:03
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Introduces a reusable FeedbackPopover component and integrates it into the Hub top bar, along with design token additions to Tailwind.

  • Adds FeedbackPopover with a dropdown of feedback actions and a dropdown icon.
  • Integrates FeedbackPopover into the Hub top bar (replacing the previous outline Button).
  • Extends Tailwind theme tokens (colors and border radii).

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
packages/status-network/tailwind.config.ts Adds new color tokens (including blue-50 and neutral scale variants) and larger border radii.
packages/status-network/src/components/index.tsx Exports FeedbackPopover from the components package.
packages/status-network/src/components/feedback-popover/index.tsx Implements the FeedbackPopover using DropdownMenu with mapped feedback links.
packages/status-network/src/components/feedback-popover/dropdown-icon.tsx Adds the dropdown chevron-in-circle SVG icon for the popover button.
apps/hub/src/app/_components/top-bar.tsx Replaces the “Share feedback” Button with the new FeedbackPopover and updates imports.
.changeset/soft-vans-suffer.md Records package version bumps for status-network and hub.

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jkbktl jkbktl requested review from a team, felicio and jinhojang6 October 5, 2025 20:54
@jkbktl jkbktl force-pushed the feat/wallet-connect branch from 99fde63 to f2e7acf Compare October 7, 2025 15:52
@jkbktl jkbktl deleted the branch status-im:main October 16, 2025 11:40
@jkbktl jkbktl closed this Oct 16, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in Web & User Interfaces Oct 16, 2025
@jkbktl jkbktl reopened this Oct 16, 2025
@github-project-automation github-project-automation bot moved this from Done to Backlog in Web & User Interfaces Oct 16, 2025
@jkbktl jkbktl moved this from Backlog to In Progress in Web & User Interfaces Oct 16, 2025
@jkbktl jkbktl changed the base branch from feat/wallet-connect to main October 16, 2025 11:51
@status-im-auto
Copy link
Member

status-im-auto commented Oct 20, 2025

Jenkins Builds

Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ dbc84a3 #7 2025-10-20 14:25:29 ~43 sec connector 📦zip
✔️ dbc84a3 #7 2025-10-20 14:26:03 ~1 min wallet 📦zip

@jkbktl jkbktl force-pushed the feat/feedback-actions branch from dbc84a3 to 4488632 Compare October 20, 2025 14:28
@status-im status-im deleted a comment from vercel bot Oct 20, 2025
Copy link

@JulesFiliot JulesFiliot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@github-project-automation github-project-automation bot moved this from In Progress to Waiting in Web & User Interfaces Oct 21, 2025
@jkbktl jkbktl merged commit 062c190 into status-im:main Oct 22, 2025
3 of 8 checks passed
@github-project-automation github-project-automation bot moved this from Waiting to Done in Web & User Interfaces Oct 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

add feedback actions

5 participants